-
Notifications
You must be signed in to change notification settings - Fork 748
feat(amazonq): removing scroll bar and adding text wrapper for commands #7143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
| margin: 0; | ||
| padding: 0; | ||
| --mynah-font-family: var(--vscode-font-family), system-ui, -apple-system, BlinkMacSystemFont, 'Segoe UI', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we move these to a dedicated css file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes we can do that. Maybe it chat_webview.css?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You'll probably have to follow: https://github.com/aws/aws-toolkit-vscode/blob/master/packages/core/src/amazonq/webview/generators/webViewContent.ts#L73 and allowlist the css in the webview that should be a good resource though if you haven't seen it yet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can I add a TODO and come back to it later?
packages/amazonq/.changes/next-release/Feature-89cee35a-a2c0-4b10-b492-4e9c5b4d904b.json
Outdated
Show resolved
Hide resolved
|
Problem
When the command is long, customers need to scroll right for more contents, which is inconvenient.
Solution
Removing scroll bar and adding text wrapper for commands. Also copying all custom styles from https://github.com/atonaamz/aws-toolkit-vscode/blob/feature/agentic-chat/packages/core/resources/css/amazonq-webview.css.
Before and after the change:


feature/xbranches will not be squash-merged at release time.